Skip to content

feat(flink): implement suggestion support for table properties in CREATE TABLE#417

Merged
mumiao merged 11 commits into
DTStack:nextfrom
ekazakas:next
May 16, 2025
Merged

feat(flink): implement suggestion support for table properties in CREATE TABLE#417
mumiao merged 11 commits into
DTStack:nextfrom
ekazakas:next

Conversation

@ekazakas

@ekazakas ekazakas commented May 5, 2025

Copy link
Copy Markdown
Contributor

Implemented SQL suggestion support for Flink table properties to work with ANTLR C3.

Related conversation: #391

Previous PR: #392 closed due to a mess with resolving conflicts

@ekazakas ekazakas changed the title Next feat(flink): implement suggestion support for table properties in CREATE TABLE May 5, 2025
@ekazakas

ekazakas commented May 5, 2025

Copy link
Copy Markdown
Contributor Author

@JackWang032 @Cythia828 @liuxy0551 I have closed the #392 since it was based from main branch and caused quite a bit of conflicts. It was much simpler to branch off the next branch and cherry pick my changes.

@mumiao mumiao requested review from JackWang032 and liuxy0551 May 8, 2025 06:07
@mumiao mumiao added the improvement Improve existing feature label May 8, 2025
@mumiao

mumiao commented May 8, 2025

Copy link
Copy Markdown
Collaborator

ci failed @ekazakas

Comment thread test/parser/flink/suggestion/tokenSuggestion.test.ts Outdated
Comment thread test/parser/flink/suggestion/tokenSuggestion.test.ts Outdated
Comment thread test/parser/flink/suggestion/fixtures/tokenSuggestion.sql Outdated
@mumiao

mumiao commented May 16, 2025

Copy link
Copy Markdown
Collaborator

+1

@mumiao mumiao merged commit 297b2f9 into DTStack:next May 16, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

improvement Improve existing feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants